Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(progress): 5770 - Adding value for accessibility attributes into … #5819

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

antoinefamibelle
Copy link

Pull Request

This pull request addresses #5770.

Changes

  • Default Style & New-York Style: ✅
    • I've made modifications for both default and New-York styles.
  • Registry: ✅
    • I've built the registry.
  • Documentation: No changes needed.
    • This modification is completely transparent to the user.
  • Screenshot: I've added screenshot where we can see the attributes in the html for both styles.
    • The component now has the desired accessibility attributes.

Don't hesitate to reach out to me if anything is wrong about it.

Screenshot 2024-11-13 at 16 40 25 Screenshot 2024-11-13 at 16 40 14

Copy link

vercel bot commented Nov 13, 2024

Someone is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

@@ -11,7 +11,7 @@ export const Index: Record<string, any> = {
type: "registry:ui",
registryDependencies: undefined,
files: [{
path: "/Users/shadcn/Code/shadcn/ui/apps/www/registry/new-york/ui/accordion.tsx",
path: "/Users/antoinefamibelle/Develop/shadcn-ui/apps/www/registry/new-york/ui/accordion.tsx",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

careful, seems your pnpm build:registry mightve changed some unintended things in index.tsx here

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In fact ! I pushed a fix sorry for the incovenience !

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm still seems to of changed something, not sure the best way to revert this type of registry change

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes my bad I've removed it from the commit completly sorry for the inconvenience. 👍

@antoinefamibelle antoinefamibelle force-pushed the fix/5770-progress-bar-accessibility-attributes branch from b10ffec to ae8d676 Compare November 14, 2024 01:44
@antoinefamibelle antoinefamibelle force-pushed the fix/5770-progress-bar-accessibility-attributes branch from ae8d676 to 436ab12 Compare November 14, 2024 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants